Skip to content

feat(packages): add packages config to very_good.yaml - #1663

Merged
marcossevilla merged 5 commits into
mainfrom
vgv-ai-bot/issue-1656
Jul 27, 2026
Merged

feat(packages): add packages config to very_good.yaml#1663
marcossevilla merged 5 commits into
mainfrom
vgv-ai-bot/issue-1656

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Adds support for configuring the very_good packages get command through the very_good.yaml configuration file.

What Changed

  • Added VeryGoodPackagesConfig and VeryGoodPackagesGetConfig classes to support configuration of the packages command
  • Added PackagesGetOptions class to resolve command-line arguments against configuration file values
  • Updated PackagesGetCommand to load and apply configuration from very_good.yaml
  • Added argument resolution logic that respects CLI precedence: CLI args > config file > CLI defaults
  • Updated documentation in get_pkgs.md with configuration examples
  • Added comprehensive tests for configuration parsing and precedence

Configuration

Users can now configure defaults in very_good.yaml:

packages:
  get:
    recursive: true
    ignore:
      - example

Closes #1656

@unicoderbot @marcossevilla
feat(packages): add packages config to very_good.yaml
def023e 
Closes #1656

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot
unicoderbot Bot requested a review from a team as a code owner July 27, 2026 10:58
@unicoderbot unicoderbot Bot mentioned this pull request Jul 27, 2026
Closed
2 tasks
@unicoderbot

unicoderbot Bot commented Jul 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

CI failed on this PR. I'm looking into it and will push a fix (attempt 1 of 3).

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Inspecting CI failures
  • Working on fix
  • Pushing fix

I've pushed a fix for the failing CI (attempt 1/3). Re-run CI to confirm.

ryzizub
ryzizub approved these changes Jul 27, 2026
View reviewed changes

@ryzizub ryzizub left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@marcossevilla
marcossevilla merged commit a4505cf into main Jul 27, 2026
24 checks passed
@marcossevilla
marcossevilla deleted the vgv-ai-bot/issue-1656 branch July 27, 2026 14:29
@vgvbot vgvbot mentioned this pull request Jul 27, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcossevilla marcossevilla marcossevilla approved these changes
@ryzizub ryzizub ryzizub approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat(packages): add packages config to very_good.yaml

2 participants

@marcossevilla @ryzizub